Skip to content

Conversation

ivancea
Copy link
Contributor

@ivancea ivancea commented Jul 9, 2025

Backports the following commits to 8.17:

Fixes elastic#130770

Both a bigger thread pool and not draining the semaphore permits were leading to failing tests sometimes because of blocked threads (Too many threads searching would end up draining all permits in parallel, and getting stuck).

The test was added in elastic#130452
@ivancea ivancea added :Analytics/Aggregations Aggregations >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Jul 9, 2025
@elasticsearchmachine elasticsearchmachine merged commit 0c923e2 into elastic:8.17 Jul 9, 2025
16 checks passed
@ivancea ivancea deleted the backport/8.17/pr-130810 branch July 9, 2025 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Aggregations Aggregations auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.17.10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants